Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Go Report Card #250

Closed
wants to merge 1 commit into from
Closed

Add Go Report Card #250

wants to merge 1 commit into from

Conversation

caniszczyk
Copy link
Contributor

@caniszczyk caniszczyk self-assigned this Oct 14, 2016
@wking
Copy link
Contributor

wking commented Oct 15, 2016

I don't know what's going on with the Travis failure; it seems like
$TRAVIS_COMMIT_RANGE is unset 1. Travis has had problems with that
in the past (travis-ci/travis-ci#1719), but they claim to have fixed
them. Maybe we just need to kick the tests?

c71bd20 looks good to me.

@Mashimiao
Copy link

@caniszczyk Do you really want to create a new branch of this project?
I think you should create branch on you own fork and create PR with that branch, not like this.

@liangchenye
Copy link
Member

@Mashimiao I guess you miss read the branch.

@Mashimiao
Copy link

No, I think I didn't. A new branch add-go-report-card has been created and pushed to this project.

@liangchenye
Copy link
Member

Oh yes, @Mashimiao I did not notice this :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants